Skip to content
This repository has been archived by the owner on Apr 14, 2021. It is now read-only.

Added jQuery plugin method #77

Merged
merged 3 commits into from
Jan 5, 2018
Merged

Added jQuery plugin method #77

merged 3 commits into from
Jan 5, 2018

Conversation

iamphill
Copy link
Owner

@iamphill iamphill commented Jan 5, 2018

Closes #73

# Transform checker
#
# Checks if transform3d is available for us to use
_transformSupported: =>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unnecessary fat arrow.

.closest('.active')
.removeClass 'active'
.closest('.open')
.removeClass 'open'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line contains inconsistent indentation. Expected 4 got 14.

@@ -24,8 +24,8 @@

# Hide currently visible dropdown menus
$('.dropdown-toggle').not(@element)
.closest('.active')
.removeClass 'active'
.closest('.open')

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line contains inconsistent indentation. Expected 4 got 2.

@iamphill iamphill merged commit a91d120 into master Jan 5, 2018
@iamphill iamphill deleted the jquery-plugin branch January 5, 2018 18:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants